home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part1 / 2440 < prev    next >
Encoding:
Internet Message Format  |  1996-08-06  |  2.6 KB
  1. Path: newsroom.hitc.com!usenet
  2. From: psand@eos.hitc.com (G. Patrick Sand)
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: If statements, HELP!!
  5. Date: 17 Jan 1996 20:09:25 GMT
  6. Organization: Hughes Aircraft (EOSDIS)
  7. Message-ID: <4djl1l$hrh@newsroom.hitc.com>
  8. References: <956_9601132330@medtechnet.com>
  9. NNTP-Posting-Host: 155.157.118.56
  10. Mime-Version: 1.0
  11. X-Newsreader: WinVN 0.99.3
  12.  
  13. In article <956_9601132330@medtechnet.com>, 
  14. Tony.Johnston@dreams.medtechnet.com says...
  15. >
  16. >Can anyone see in this code fragment why this if statement wont do what
  17. >I need it to do.  Using Borland C++ 4.0:
  18. > 
  19. > 
  20. >{
  21. > 
  22. >    if (SysopPercent > UserPercents)  <--If this is true it should fall 
  23. down 
  24. >       {                                 to the next if, if false it 
  25. should
  26. >                                         go to the ELSE
  27. >       if (SysTime < LockTime)        <--If true it should process its
  28. >                                         statement, if false it should 
  29. fall
  30. >                                         down to the else
  31. >       TDDisplayFile("LOCK.ANS");
  32. >       }
  33. >       else
  34.  
  35. 1. The else is only hit when SysopPercent <= UserPercents !!! not 
  36.    SysTime >= LockTime
  37. 2. Only the first call is done (TBDoors_Tmp.Open())...
  38.  
  39. >        TBDoors_Tmp.Open();
  40.  
  41. 3. The rest is always called because it is not enclosed in a brackets 
  42. block...
  43.  
  44. >        TBDoors_Tmp.Read();
  45. >        TBDoors_Tmp.WhereTheDoorWasCalledFrom(5);
  46. >        TBDoors_Tmp.Write();
  47. >        TBDoors_Tmp.Close();
  48. > 
  49. > 
  50. >}
  51. [snip!]
  52.  
  53. I BET YOU WERE EXECUTING THE TBDoors_Tmp.Open() call when the percentage 
  54. test failed...run the debugger and step through this logic to verify...
  55.  
  56. Recode along these lines:
  57.  
  58. // indent and align as you like...
  59.  
  60. // if it fails either condition, do the "ELSE" stuff...
  61. if ( ( SysopPercent <= UserPercents ) || ( SysTime >= LockTime ) )
  62. {
  63.         TBDoors_Tmp.Open();
  64.         TBDoors_Tmp.Read();
  65.         TBDoors_Tmp.WhereTheDoorWasCalledFrom(5);
  66.         TBDoors_Tmp.Write();
  67.         TBDoors_Tmp.Close();
  68. } else
  69. {
  70. // it passed both conditions so do the desired thing...
  71.      TDDisplayFile("LOCK.ANS");
  72. };
  73.  
  74.  
  75. I like to put brackets in the "THEN" and "ELSE" parts of the code even 
  76. when they are single lines to avoid such problems as these...
  77.  
  78. That should fix it, if I understood things correctly...
  79. -- 
  80. G. Patrick Sand
  81. psand@eos.hitc.com
  82. PatSand@aol.com
  83. (301) 925-0791
  84. "Travel Light But Right..."
  85. Microsoft Network is prohibited from redistributing 
  86. this work in any form, in whole or in part.   License 
  87. to distribute this individual post is available to Microsoft
  88. for $999. Posting without permission constitutes an 
  89. agreement to these terms...gps
  90.  
  91.